Skip to content

Add [tool.uv] exclude-newer config to arcade new's full template#809

Merged
EricGustin merged 3 commits intomainfrom
ericgustin/full-template-uv-exclude
Apr 8, 2026
Merged

Add [tool.uv] exclude-newer config to arcade new's full template#809
EricGustin merged 3 commits intomainfrom
ericgustin/full-template-uv-exclude

Conversation

@EricGustin
Copy link
Copy Markdown
Member

@EricGustin EricGustin commented Apr 1, 2026

templates the changes in https://github.com/ArcadeAI/monorepo/pull/765


Note

Low Risk
Low risk: only updates template/configuration and a patch version bump, with no runtime logic changes.

Overview
Updates the arcade new full template pyproject.toml to include a [tool.uv] section that sets exclude-newer = "1 week" while explicitly disabling this constraint for core Arcade packages via exclude-newer-package.

Bumps the root arcade-mcp package version from 1.13.2 to 1.13.3.

Reviewed by Cursor Bugbot for commit 579cf41. Bugbot is set up for automated code reviews on this repo. Configure here.

@EricGustin EricGustin changed the title Add [tool.uv] exclude-newer config to full template and bump CLI to 1.12.3 Add [tool.uv] exclude-newer config to arcade new's full template Apr 1, 2026
@EricGustin EricGustin requested review from byrro and jottakka April 1, 2026 20:25
@codecov
Copy link
Copy Markdown

codecov bot commented Apr 1, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Copy link
Copy Markdown

@cursor cursor bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Autofix Details

Bugbot Autofix prepared a fix for the issue found in the latest run.

  • ✅ Fixed: Invalid boolean value for exclude-newer-package setting
    • Replaced the invalid boolean value with a far-future date string (2099-12-31) to effectively exempt arcade-mcp-server from the exclude-newer constraint.

Create PR

Or push these changes by commenting:

@cursor push 02427cd6fe
Preview (02427cd6fe)
diff --git a/libs/arcade-cli/arcade_cli/templates/full/{{ toolkit_name }}/pyproject.toml b/libs/arcade-cli/arcade_cli/templates/full/{{ toolkit_name }}/pyproject.toml
--- a/libs/arcade-cli/arcade_cli/templates/full/{{ toolkit_name }}/pyproject.toml
+++ b/libs/arcade-cli/arcade_cli/templates/full/{{ toolkit_name }}/pyproject.toml
@@ -55,7 +55,8 @@
 
 [tool.uv]
 exclude-newer = "1 week"
-exclude-newer-package = { arcade-mcp-server = false }
+# Exempting arcade-mcp-server from the exclude-newer constraint
+exclude-newer-package = { arcade-mcp-server = "2099-12-31" }
 
 [tool.hatch.build.targets.wheel]
 packages = ["{{ package_name }}"]

This Bugbot Autofix run was free. To enable autofix for future PRs, go to the Cursor dashboard.


[tool.uv]
exclude-newer = "1 week"
exclude-newer-package = { arcade-mcp-server = false }
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Invalid boolean value for exclude-newer-package setting

Medium Severity

The exclude-newer-package setting in uv accepts per-package timestamp values (RFC 3339 timestamps or local dates like "2006-12-02"), not booleans. Setting { arcade-mcp-server = false } is likely an invalid value that will cause a uv configuration parsing error when users try to resolve or install dependencies in a generated toolkit project. The intent to exempt arcade-mcp-server from the exclude-newer constraint may not be achievable with this syntax.

Fix in Cursor Fix in Web

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copy link
Copy Markdown

@cursor cursor bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes and found 1 potential issue.

There are 2 total unresolved issues (including 1 from previous review).

Fix All in Cursor

Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.


[tool.uv]
exclude-newer = "1 week"
exclude-newer-package = { arcade-mcp-server = false, arcade-tdk = false, arcade-core = false, arcade-serve = false }
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Missing arcade-mcp from exclude-newer-package exclusion list

Medium Severity

The exclude-newer-package list excludes arcade-mcp-server, arcade-tdk, arcade-core, and arcade-serve from the 1-week exclude-newer restriction, but omits arcade-mcp, which is a direct dev dependency of the template (via arcade-mcp[evals] on line 26). Since arcade-mcp is also an internal Arcade package, it needs the same exemption. Without it, newly generated toolkits may fail to resolve recently published versions of arcade-mcp.

Fix in Cursor Fix in Web

Copy link
Copy Markdown
Member

@byrro byrro left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@EricGustin EricGustin merged commit 987a4ea into main Apr 8, 2026
31 checks passed
@EricGustin EricGustin deleted the ericgustin/full-template-uv-exclude branch April 8, 2026 18:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants